Skip to content

Add Codex plugin#177

Open
keertip wants to merge 3 commits into
flutter:mainfrom
keertip:codex-plugin
Open

Add Codex plugin#177
keertip wants to merge 3 commits into
flutter:mainfrom
keertip:codex-plugin

Conversation

@keertip

@keertip keertip commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Add configuration files for a Codex plugin.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds configuration files for the 'dart-flutter' plugin to support Codex, including marketplace and plugin metadata, and updates the README with installation instructions. The review feedback suggests adding a blank line before the new '### Codex Plugin' heading in the README to ensure proper Markdown rendering.

Comment thread README.md Outdated
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@keertip keertip marked this pull request as ready for review July 16, 2026 13:41
@keertip keertip requested a review from kenzieschmoll July 16, 2026 13:42
Comment thread .codex-plugin/plugin.json Outdated
Comment thread .codex-plugin/plugin.json
Comment on lines +34 to +35
"Set up Flutter in this app.",
"Create a new Dart skill."

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we have default prompts for the other plugins? And if so are these the ones we want? CC @mariamhas

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The other plugin manifests don't have this field. Should we just not have these prompts? We are modeling this based on the Firebase plugins.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm i say we leave it out.. we should test to see how this plugin works and if/what prompts it needs to be helpful in using skills and mcp that are part of the plugin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants